colorchooser: Fix a crash

Don't crash when we start removing custom colors
that happen to be currently selected.

emojichooser: Drop focus adjustment setting

It is not needed here, we are now scrolling
differently.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #1899 and #2627

See merge request GNOME/gtk!1709

Merge branch 'ebassi/for-master' into 'master'

cups: Disable deprecation warnings

See merge request GNOME/gtk!1710

migration docs: Updates

Mention shadow type, relief, aspect frame.

cups: Disable deprecation warnings

It seems newer releases of CUPS removed the ability to disable
deprecation warnings by defining a pre-processor symbol, so we
have to resort to the usual begin/end ignore deprecation pragmas
to avoid a ton of deprecation warnings.

Merge branch 'for-master' into 'master'

For master

See merge request GNOME/gtk!1708

aspectframe: Modernize

Make GtkAspectFrame not derive from GtkFrame anymore,
since frames now always draw, well, a frame. Also,
add proper setters for the properties of GtkAspectFrame.

Update our sole user.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2627

Reconcile documentation with declaration and definition

The arguments name in the documentation must match declaration and
definition, or gtk-doc and g-ir-scanner will get very cross at us.

Copy axes when rewriting events

Adwaita: Add focus to expander

We were not drawing a focus indication at all for
expanders. Bring it back. Also, make the color plane
focus snug.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/1899

nodeeditor: Don't set has-focus

That is now a readonly property. Set focus-widget
on the toplevel instead.

Merge branch 'menu-section-title' into 'master'

popovermenu: Reinstate section title handling

See merge request GNOME/gtk!1707

Merge branch 'matthiasc/for-master' into 'master'

gsk: Fall back to cairo if compiling shaders fails

See merge request GNOME/gtk!1706

popovermenu: Reinstate section title handling

We were creating a hbox with separators around the label,
but forgot to arrange sizing to prevent the separators from
shrinking into nothingness.

gsk: Fall back to cairo if compiling shaders fails

There was already some fallback in place here, but
in other places we were just asserting.

Merge branch 'matthiasc/for-master' into 'master'

gsk: Fall back to cairo if compiling shaders fails

See merge request GNOME/gtk!1705

widget-factory: Add a menu section title

If you can't see it, you can't theme it.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1703

reftests: Stop using shadow-type

calendar: Fix focus keynav

This broke when the calendar was broken into widgets.

Drop GtkReliefStyle

We are not using this enum anymore.

menubutton: Drop relief

We are only using this as a boolean, so change it
to a boolean property named has-frame.

button: Drop relief

We are only using this as a boolean, so change it
to a boolean property named has-frame.

Drop GtkShadowType

We are not using this enum anymore.

viewport: Drop shadow-type

The viewport draws a frame at the same place as
the scrolled window, so there is really no need
to have that ability in both. Just drop the frame
from viewports.

frame: Drop shadow-type

Frames that don't draw frames are not very useful,
so just drop the shadow-type property.

treeviewcolumn: Stop using a frame

There is no desire to draw a frame here, so don't use a GtkFrame.

scrolledwindow: Drop shadow-type

We were only using this as a boolean, so change it
to a boolean property named has-frame.

scrolledwindow: Document the .frame style class

Revert "modelbutton: Emit ::clicked() before closing the popover"

This reverts commit 57f913b7533b3b27ecd17ed7629e2b473e89bcbd.

Merge branch 'assorted-menu-fixes' into 'master'

Assorted menu fixes

See merge request GNOME/gtk!1704

snapshot: Avoid a save/restore pair if we can

label: Ignore setting attrs from NULL to NULL

text: unconditionally unref attr list

pango_attr_list_unref() works on NULL

text: Avoid creating a PangoAttrList we don't need

It's going to be empty, unless we have preedit_attrs

meson: Use summary()

Instead of doing our own. Bumps the dependency to 0.53 though.

meson: Trivial formattting

label: Redraw when text attrs are affected by css change

GtkWidget doesn't do it automatically and probably shouldn't

label: Don't create an unnecessary attribute list

label: Only parse mnemonic attributes if we need to

label: Ignore AFFECTS_ATTRS css changes if we can

It's only interesting if we either already have attributes, or the new
style adds some.

label: Inline function into only caller

label: Remove some unused quarks

label: Remove priv pointer

label: Stop connecting to notify::gtk-enable-accels

Leftover from when GtkAccelLabel was a GtkLabel subclass. GtkAccelLabel
connects to this itself these days though.

label: Set underline text and markup in one step

Try to unify the way we parse the mnemonic character

label: Fold set_pattern_internal into only caller

adwaita: don't extent %link in linkbuttons

We already do that for the label inside the button.

label: Remove GtkLabel:track-visited-links

Always track visited state of links. This way all visited links in all
labels look the same. Whether the theme wants to style :visited is the
theme's business.

label: Remove a11y-only properties

Update them manually instead.

label: Fold function into only caller

This makes it more obvious that gtk_label_setup_mnemonic() depends on
the root of the widget and it therefore makes sense to call it in
::root/::unroot.

label: Don't normalize booleans in internal setters

We almost always pass a literal TRUE/FALSE to them.

stylecontext: Remove get_pango_attributes

Make the only caller use GtkCssNode API.

cssstyle: Only create PangoAttrList if there are attribtues

Don't allocate a new GString if we never need it and therefore don't
create the PangoAttrList if we have no attributes anyway. Update callers
to handle the possible NULL return value.

label: Pull all pango attributes from the theme

Dont' force an underline attribute here. Just ask the theme for the
text-decoration.

label: Fix an old TODO comment

modelbutton: Emit ::clicked() before closing the popover

The clicked handler might still need the popover.

label: Remove public pattern API

label: Shorten set_markup_internal a bit

label: Avoid duplicating a string

We only use str_for_accel if with_uline is TRUE.

label: Pass <markup> length directly when parsing markup

label: Remove a double if (needs_root) check

The code is clearer and shorter this way.

label: Use widget API to add style class

label: Clear select info before creating a new one

E.g. set_markup_internal will create a new select_info, but then we were
destroying it again, just to recreate it later.

label: Save a few LOC

label: Save links in an array

Stop using GList for this.

widget: Return an array from list_devices

To forther reduce the GList usage in the code base.

accelgroup: Restructure gtk_accelerator_name

To fix invalid reads and make the function a bit shorter while we're at
it.

Fixes #2602

main: Stop using GList when propagating events

There are still some cases missing.

widget-factory: Use a model for the complex menu

We had the model already in the ui file, but weren't
using it.

modelbutton: Don't show accels without text

We don't want to show accelerators in iconic buttons.

widget-factory: Catch errors

This speeds up the debugging when doing quick experimental
changes to widget-factory.ui.

inspector: Fix a crash

Don't crash while picking. The event signal broke
when GdkEvent was turned into a type instance,
since the automatic marshallers don't know how to
deal with that. Manually set the right marshaller.

Merge branch 'event-types' into 'master'

Rework event types

Closes #2585

See merge request GNOME/gtk!1702

Restructure the GdkEvent type hierarchy

GdkEvent has been a "I-can't-believe-this-is-not-OOP" type for ages,
using a union of sub-types. This has always been problematic when it
comes to implementing accessor functions: either you get generic API
that takes a GdkEvent and uses a massive switch() to determine which
event types have the data you're looking for; or you create namespaced
accessors, but break language bindings horribly, as boxed types cannot
have derived types.

The recent conversion of GskRenderNode (which had similar issues) to
GTypeInstance, and the fact that GdkEvent is now a completely opaque
type, provide us with the chance of moving GdkEvent to GTypeInstance,
and have sub-types for GdkEvent.

The change from boxed type to GTypeInstance is pretty small, all things
considered, but ends up cascading to a larger commit, as we still have
backends and code in GTK trying to access GdkEvent structures directly.
Additionally, the naming of the public getter functions requires
renaming all the data structures to conform to the namespace/type-name
pattern.

Fix annotations for GtkEventController

Fix annotation for gtk_show_uri_full()

The callback is now an asynchronous closure.

Fix compiler warning

Newer versions of GCC are fairly aggressive with NULL checks.

Merge branch 'matthiasc/for-master' into 'master'

themes: Fix HighContrast dependencies

See merge request GNOME/gtk!1701

Merge branch 'css-focus-changes' into 'master'

Add a focus-within state

See merge request GNOME/gtk!1700

themes: Fix HighContrast dependencies

HighContrast themes now depend on sass files from Adwaita.

Adwaita: Some adjustments for :focus-within

Use :focus-within for focus in entries, since the
actual focus is on the text within, and :focus for
notebooks, since we don't want to draw an outline
around the notebook when the focus is in content.

testsuite: Add some checks for focus states

This is some opportunistic checking in a test that
is really about focus chains, but better than nothing.

Add a focus-within state

This is used for widgets that contain the focus widget,
reserving the focused state for the focus location itself.

This aligns our focus state handling with
https://www.w3.org/TR/selectors-4/

Merge branch 'monitor-signals-x11' into 'master'

x11: emit ::enter/leave-monitor

See merge request GNOME/gtk!1699

x11: emit ::enter/leave-monitor

For the X11 backend, keep a list of monitors for which the surface
intersects the monitor area.

Whenever the X11 surface is configured, check against the list of
monitors to determine whether it enters a new monitor or if it left a
monitor, to emit the corresponding ::enter/leave-monitor signals just
like a Wayland compositor would.

As monitors can be added, removed or reconfigured at any time, redo
those checks whenever any of these events occur.

Add :focus-visible to css docs

css: use :focus-visible instead of :focus(visible)

This is aligning our language with
https://www.w3.org/TR/selectors-4/

Merge branch 'matthiasc/for-master' into 'master'

window: Don't set is-active twice

See merge request GNOME/gtk!1698

window: Don't set is-active twice

We are listening to focus change events for this
no need to duplicate the work in ::state-changed.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #2612

See merge request GNOME/gtk!1695

Updates to NEWS

Move key event rewriting

Stop rewriting key and focus events on the GDK side.
Instead deliver them as they are, and propagate them
from the root on the gtk side, in gtkmain.c. And
stop complaining about focus events on popups - we
can just ignore them if we have no use for them.

window: Fix setting :is-active

We want to listen to focus events from the windowing
system here, relying on crossing events for this doesn't
work.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2612

eventcontroller: Fix signal marshallers

GdkEvent is no longer a GObject. Any signals taking
events as arguments need to have their marshallers
fixed. Thankfully, we have few of these left.

Merge branch 'wip/jimmac/drop-text-shadow' into 'master'

Adwaita: drop text shadows for buttons

Closes #2280

See merge request GNOME/gtk!1693

Adwaita: drop text shadows for buttons

- People seem to misunderstand the unsharp-mask-like increase
  of visual contrast for "fuziness". That is not the reason for
  the change. The stylistic change of just going with flat
  text label allows to simplify the code and drop complexity.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2280

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1694

Merge branch 'wip/jimmac/checkradio-focus' into 'master'

Adwaita: check & radio focus ring

See merge request GNOME/gtk!1690